Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: 공용 스타일 파일 디테일 수정 #19

Merged
merged 4 commits into from
Dec 20, 2024

Conversation

minim1nh
Copy link
Collaborator

📝 Description

asset 명 수정 및 variables 파일 수정

📸 Screenshot

스크린샷 2024-12-20 오후 9 25 57

📢 Notes

@minim1nh minim1nh self-assigned this Dec 20, 2024
@minim1nh minim1nh added the ⚙ Setting 프로젝트 설정이나 환경 설정과 관련된 경우 label Dec 20, 2024
Copy link
Collaborator

@changhui-chan changhui-chan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@minim1nh minim1nh merged commit 92418ee into develop Dec 20, 2024
2 checks passed
Copy link
Owner

@21ow 21ow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐧님 언더바 때문에 이틀 가량 고생하신 것 같아요...
😂😂 감사합니다. 고생 많으셨어요!
나머지 파일은 제가 언더바 제거해서 올리겠습니다.

@import url('https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/variable/pretendardvariable.min.css');
@import '@/_asset/reset.scss';
@import '@/asset/reset.scss';
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✋질문 있습니다.
기존 방식이 globals에서 reset, variables를 import해서 관리하는 방식이었잖아요?
variables는 config에서 전역으로 사용 가능하게 바꾸신 거고요.
reset도 config에 추가해서 사용하는 게 가능할까요??
어차피 variables처럼 전역으로 사용되는 파일이니까??

$radi-category-small: 6px;
$radi-category-filter: 100px;
$radi-modal: 16px;
$radius-common: 8px;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

섬세함 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙ Setting 프로젝트 설정이나 환경 설정과 관련된 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants